-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update aiida-core to 2.6.2 #474
Conversation
I think @sphuber will firing another patch release? Let's wait a bit for that.
Let's hold a bit on this. For PH calculation and bands calculation, we are definitely not ready to move forward on this. But I think we need to start testing on it more, to make the problem more exposed. If you are interested, please have a look at the AEP (aiidateam/AEP#35) and the initial discussion (aiidateam/aiida-core#5178). I had some workarounds with changing the workchain logic to catch the remote data clean up issue to rerun the parent calcualtion, but far from perfect. I'll update the AEP and make a slide to try to describe the problem more clear. |
a31d1c7
to
77379d4
Compare
77379d4
to
052fedc
Compare
052fedc
to
7397caa
Compare
Aiida 2.6.2 has been released today. 🎉 This should be ready to go. The new @unkcpz @superstar54 LMK if you're generally available for reviews or if I should be reaching out to somebody else in the meantime. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @danielhollas, I think it is ready to go.
@danielhollas @unkcpz I'm unable to install aiida-core>2.5.1 on aiidalab-fullstack containers.
|
Since we had pinned the |
I suspected as much and pulled the new version. Same issue 😭 |
Please try pull |
Looks like we're good to go here. I've launched tests for AWB in aiidalab/aiidalab-widgets-base#616 and for QeApp in aiidalab/aiidalab-qe#760 and they all passed.
Once we update, I think we can enable caching by default, per #407.